Update packaging, upgrade to TypeScript 1.8 #33
Conversation
kitsonk
commented
Feb 24, 2016
- Migrate to npm only for dependencies
- Add typings to package.json for distribution builds
- Remove the _modules path
- Update form.ts to be compatible with TS 1.8
- Update other npm dependencies to latest versions
- Amend tslint.json to be compatible latest version
- Migrate to npm only for dependencies - Add typings to package.json for distribution builds - Remove the _modules path - Update form.ts to be compatible with TS 1.8 - Update other npm dependencies to latest versions - Amend tslint.json to be compatible latest version
///<reference path="../_modules/dojo-core/typings/dojo-core/dojo-core-2.0.0-pre.d.ts"/> | ||
///<reference path="../node_modules/dojo-core/typings/dojo-core/dojo-core-2.0.0-alpha.2.d.ts"/> | ||
/// <reference path="../node_modules/dojo-loader/typings/dojo-loader/dojo-loader-2.0.0-alpha.4.d.ts" /> | ||
/// <reference path="../tests/typings/node/node.d.ts" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
missing new line EOF
Looks good 👍 |
@@ -1 +1,3 @@ | |||
///<reference path="../_modules/dojo-core/typings/dojo-core/dojo-core-2.0.0-pre.d.ts"/> | |||
///<reference path="../node_modules/dojo-core/typings/dojo-core/dojo-core-2.0.0-alpha.2.d.ts"/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing space after ///
Current coverage is
|
@vansimke have you reviewed this? If so, if you could add a comment to that effect. |
Sorry, haven't looked at it yet. Didn't realize this was a PR, so I didn't look for it. I'll take a look this morning. |
This looks good to me |
}, | ||
"devDependencies": { | ||
"codecov.io": "0.1.6", | ||
"dojo-loader": "2.0.0-alpha.1", | ||
"dojo-loader": ">2.0.0-alpha", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-beta2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is still valid semver for beta.2